Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mpl: RIP TritonMacroPlace (mpl) #6047

Merged

Conversation

openroad-robot
Copy link
Contributor

Replaced by Hierarchical RTL-MPL (mpl2; to be renamed mpl)

Replaced by Hierarchical RTL-MPL (mpl2; to be renamed mpl)

Signed-off-by: Matt Liberty <[email protected]>
@gadfort
Copy link
Collaborator

gadfort commented Oct 29, 2024

@maliberty can you hold off on merging this until I've verified this won't cause a whole host of issues in my CI?

@maliberty
Copy link
Member

I was planning to wait a bit for feedback. I do want to drop it as we no longer do any development or support for mpl.

@gadfort
Copy link
Collaborator

gadfort commented Oct 29, 2024

Minor metrics update for tinyRocket_nangate45 for clock skew

Signed-off-by: Matt Liberty <[email protected]>
@AcKoucher
Copy link
Contributor

AcKoucher commented Jan 22, 2025

@gadfort Would it be possible to give your CI another go with the changes here now that #5809 was merged?

@gadfort
Copy link
Collaborator

gadfort commented Jan 22, 2025

@AcKoucher sure, but I would need this PR to be updated to include those changes.

@maliberty
Copy link
Member

@gadfort merged

@gadfort
Copy link
Collaborator

gadfort commented Jan 23, 2025

@AcKoucher all the ones that are active pass

@maliberty
Copy link
Member

@AcKoucher all the ones that are active pass

Does that mean this is ok to merge from your perspective?

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@maliberty maliberty merged commit 44e3614 into The-OpenROAD-Project:master Jan 25, 2025
11 checks passed
@maliberty maliberty deleted the rm-old-mpl branch January 25, 2025 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants